i2c: bcm2835: Add support for Repeated Start Condition
authorNoralf Trønnes <noralf@tronnes.org>
Fri, 23 Sep 2016 02:54:27 +0000 (04:54 +0200)
committerpopcornmix <popcornmix@gmail.com>
Mon, 20 Feb 2017 20:28:41 +0000 (20:28 +0000)
commit3d84dfe11f9198c3e613e6c68da3ae8ee67435e4
tree112936099b526c6b72df00b95a01ab9faea13131
parenta528ab2dd2a65a75a2ce8566e30508dac3a1f41e
i2c: bcm2835: Add support for Repeated Start Condition

Documentation/i2c/i2c-protocol states that Combined transactions should
separate messages with a Start bit and end the whole transaction with a
Stop bit. This patch adds support for issuing only a Start between
messages instead of a Stop followed by a Start.

This implementation differs from downstream i2c-bcm2708 in 2 respects:
- it uses an interrupt to detect that the transfer is active instead
  of using polling. There is no interrupt for Transfer Active, but by
  not prefilling the FIFO it's possible to use the TXW interrupt.
- when resetting/disabling the controller between transfers it writes
  CLEAR to the control register instead of just zero.
  Using just zero gave many errors. This might be the reason why
  downstream had to disable this feature and make it available with a
  module parameter.

I have run thousands of transfers to a DS1307 (rtc), MMA8451 (accel)
and AT24C32 (eeprom) in parallel without problems.

Signed-off-by: Noralf Trønnes <noralf@tronnes.org>
Acked-by: Eric Anholt <eric@anholt.net>
drivers/i2c/busses/i2c-bcm2835.c